-
Notifications
You must be signed in to change notification settings - Fork 672
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement uniform URL for connecting remote browsers (closes #1476) #1478
Conversation
❌ Tests for the commit 9f8d92c have failed. See details: |
❌ Tests for the commit 1723c83 have failed. See details: |
❌ Tests for the commit ffbc6eb have failed. See details: |
1 similar comment
❌ Tests for the commit ffbc6eb have failed. See details: |
✅ Tests for the commit ffbc6eb have passed. See details: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but it necessary to check the string with a techwriter
src/cli/remotes-wizard.js
Outdated
|
||
if (remoteCount) { | ||
log.hideSpinner(); | ||
|
||
var description = dedent(` | ||
Connecting ${remoteCount} remote browser(s)... | ||
Navigate to the appropriate URL from each of the remote browsers. | ||
Navigate to the following URL from each of the remote browsers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Navigate to the following URL from each remote browser.
✅ Tests for the commit 87ad77a have passed. See details: |
FPR |
\сс @AlexanderMoskovkin